Skip to content

plan: validate targets before future --since exit#7

Closed
sandover wants to merge 1 commit intomainfrom
plan/cli-since-validation
Closed

plan: validate targets before future --since exit#7
sandover wants to merge 1 commit intomainfrom
plan/cli-since-validation

Conversation

@sandover
Copy link
Owner

Summary

  • add an ergo task to fix future --since early-exit bypassing target validation
  • codify remote/local acceptance criteria and validation gates

Testing

  • not run (planning only)

Add an ergo task to close the future --since validation gap.
Why: remote --since rejection and missing pool errors are currently skipped.
How: task calls for validating target and constraints before early exit.
Acceptance criteria cover remote/local follow/duplex behavior.
Preserves existing success exit for future --since on valid local pools.
Validation gates: cargo fmt --all, cargo clippy --all-targets, just ci-fast.
Constraints: keep CLI error messaging/hints aligned with existing helpers.
Risks/tests: verify remote --since rejection still fires after refactor.
@sandover
Copy link
Owner Author

Superseded by #10 after .ergo/events.jsonl -> .ergo/plans.jsonl migration. Closing stale PR to keep planning diffs mergeable on current main.

@sandover sandover closed this Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant